Skip to content

feat(framework): add QA and communication evaluations - #512

Merged
alexsoyes merged 1 commit into
nextfrom
feat/qa-video-evidence
Aug 2, 2026
Merged

feat(framework): add QA and communication evaluations#512
alexsoyes merged 1 commit into
nextfrom
feat/qa-video-evidence

Conversation

@alexsoyes

@alexsoyes alexsoyes commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

🎯 What & why

Adds post-review UI QA with reviewer evidence, including optional WebM recordings and PR attachment. Adds a fresh-context evaluation for the project communication contract.

🛠️ How it works

🧪 How to verify

  • pnpm install --frozen-lockfile
  • node scripts/check-markdown-links.js
  • cd cli && pnpm install --frozen-lockfile && pnpm run knip:production && pnpm test

Browser QA evidence

  • aidd_docs/tasks/2026_06/2026_06_24-simplifier-creation-formation/qa/happy-path.webm — vp9, 1280×720, 13,8 s
happy-path.webm

⚠️ Heads-up

QA video runs need an application URL. Chromium downloads on the first Playwright CLI use.

✅ I certify

  • I DO CERTIFY I READ EACH LINE OF THE PULL REQUEST BECAUSE I AM A SOFTWARE ENGINEER, NOT A AI PUPPY.

@alexsoyes

Copy link
Copy Markdown
Contributor Author

@blafourcade Holà ! Je sais pas si tu le fais mais j'ai fait une passe de relecture ligne par ligne de 100% des lignes générées pour en enlever... 40% facile.

Dis-moi si ça se voit ?

@alexsoyes
alexsoyes marked this pull request as ready for review July 26, 2026 06:28
@alexsoyes
alexsoyes enabled auto-merge (squash) July 26, 2026 11:24
Comment thread plugins/aidd-dev/skills/11-browser-qa/actions/02-prepare-run.md Outdated
Comment thread plugins/aidd-dev/skills/11-browser-qa/actions/03-run-scenarios.md
Comment thread plugins/aidd-dev/skills/11-browser-qa/actions/03-run-scenarios.md
Comment thread plugins/aidd-dev/skills/11-browser-qa/references/run-scope-playwright-cli.md Outdated

## Invocation

Use the framework pin below. The current version is `0.1.17`.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pourquoi ping une version vs vérifier les capacités de l'outil si tu sais que tu es sur Playwright ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On peut le faire sauter mais avec cette version de playwright je suis sûr que ça fonctionne

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'est juste qu'on créé un point d'ancrage à faire évoluer dans le temps plus tard

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'est la recommandation NPM / sécu classique...

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je sais mais je pointe la maintenance dans le temps, peut être mettre un >= ?

Comment thread plugins/aidd-dev/skills/11-browser-qa/SKILL.md
Comment thread plugins/aidd-dev/skills/11-browser-qa/SKILL.md
@blafourcade blafourcade assigned alexsoyes and unassigned blafourcade Jul 27, 2026
@alexsoyes alexsoyes assigned blafourcade and unassigned alexsoyes Jul 28, 2026
@blafourcade blafourcade assigned alexsoyes and unassigned blafourcade Jul 28, 2026
@alexsoyes
alexsoyes requested a review from a team as a code owner July 31, 2026 07:42
Add aidd-dev browser-qa: locked executable journeys, minimal
discovery, short WebM evidence recorded per journey under the task
folder. Journeys run atomically so a failed command cannot look
successful, and cut points are validated on frames rather than
metadata.

Wire it into the surrounding flow: plan phases declare a browser test
scope, project memory carries app entry, test auth, fixtures and reset
instructions, and pull requests link every changed qa/*.webm.
@blafourcade
blafourcade force-pushed the feat/qa-video-evidence branch from 2366ffc to 7915df3 Compare August 2, 2026 05:08
@alexsoyes
alexsoyes merged commit 9b1313e into next Aug 2, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants